Skip to content

west.yml: Bump TF-M to allow FOTA upgrade #94859

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Nov 5, 2025
Merged

west.yml: Bump TF-M to allow FOTA upgrade #94859

merged 2 commits into from
Nov 5, 2025

Conversation

@nandojve
Copy link
Member

@nandojve nandojve commented Aug 22, 2025 •
edited
Loading

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Aug 22, 2025
@github-actions
Copy link

github-actions bot commented Aug 22, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
trusted-firmware-m zephyrproject-rtos/trusted-firmware-m@94691a2 zephyrproject-rtos/trusted-firmware-m@d3341a6 (main) zephyrproject-rtos/trusted-firmware-m@94691a2e..d3341a66

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-trusted-firmware-m DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 22, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 22, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

This was referenced Aug 22, 2025
Merged
Draft
@nandojve nandojve force-pushed the tfm/fix_structure_to_allow_tfm_upgrade branch from 7023409 to 8fd1a30 Compare October 28, 2025 09:27
@zephyrbot zephyrbot added the area: TF-M ARM Trusted Firmware-M (TF-M) label Oct 28, 2025
@zephyrbot zephyrbot requested review from Vge0rge and ithinuel October 28, 2025 09:28
@nandojve nandojve added this to the v4.3.0 milestone Oct 28, 2025
tomi-font
tomi-font requested changes Oct 28, 2025
View reviewed changes
Copy link
Contributor

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the commit message when updating the revision, thanks!

@tomi-font tomi-font added the bug The issue is a bug, or the PR is fixing a bug label Oct 28, 2025
@nandojve
Copy link
Member Author

nandojve commented Oct 28, 2025

Fix the commit message when updating the revision, thanks!

Fixed!

@nandojve nandojve requested a review from tomi-font October 28, 2025 13:55
@nandojve nandojve force-pushed the tfm/fix_structure_to_allow_tfm_upgrade branch from 8fd1a30 to d6c9fbe Compare November 4, 2025 07:10
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Nov 4, 2025
tomi-font
tomi-font requested changes Nov 4, 2025
View reviewed changes
Copy link
Contributor

@tomi-font tomi-font left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message is not fixed:

This pull a regression pull the fixes to allow Zephyr v3.7.1 upgrade to
v4.3.0.

Also maybe this would be worth a migration guide/release notes entry because if FOTA was broken when updating from <= v3.7 to < v4.3 then maybe now it won't work when updating from Zephyr versions 4.0-4.2 to >= 4.3?

@nandojve
west.yml: Bump TF-M to fix FOTA upgrade regression
c4f3344 
Update TF-M to include fixes restoring FOTA upgrade compatibility
between Zephyr v3.7.1 and v4.3.0. This fix allow both forward and
rollback upgrades.

Signed-off-by: BUDKE Gerson Fernando <gerson.budke@leica-geosystems.com>
@nandojve nandojve force-pushed the tfm/fix_structure_to_allow_tfm_upgrade branch from d6c9fbe to 3a78066 Compare November 4, 2025 10:18
@nandojve nandojve requested a review from tomi-font November 4, 2025 10:18
@nandojve
Copy link
Member Author

nandojve commented Nov 4, 2025

Hi @tomi-font ,

Commit message is not fixed:

This pull a regression pull the fixes to allow Zephyr v3.7.1 upgrade to
v4.3.0.

Commit message reworked.

Also maybe this would be worth a migration guide/release notes entry because if FOTA was broken when updating from <= v3.7 to < v4.3 then maybe now it won't work when updating from Zephyr versions 4.0-4.2 to >= 4.3?

Add migration note with recommendation.

@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Nov 4, 2025
tomi-font
tomi-font reviewed Nov 4, 2025
View reviewed changes
@nandojve
docs: migration-guide-4.3: Add TF-M attestation issue
658acb6 
Add documentation describing a TF-M attestation issue that prevents
firmware-over-the-air (FOTA) upgrades between affected Zephyr releases.
Recommend a direct upgrade to Zephyr v4.3.0 to avoid attestation and
upgrade compatibility issues.

Signed-off-by: BUDKE Gerson Fernando <gerson.budke@leica-geosystems.com>
@nandojve nandojve force-pushed the tfm/fix_structure_to_allow_tfm_upgrade branch from 3a78066 to 658acb6 Compare November 4, 2025 14:09
@nandojve nandojve requested a review from tomi-font November 4, 2025 14:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 4, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@jhedberg jhedberg merged commit 80be486 into zephyrproject-rtos:main Nov 5, 2025
28 checks passed
@nandojve nandojve deleted the tfm/fix_structure_to_allow_tfm_upgrade branch November 5, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jhedberg jhedberg jhedberg approved these changes

@tomi-font tomi-font tomi-font approved these changes

@valeriosetti valeriosetti Awaiting requested review from valeriosetti

@wearyzen wearyzen Awaiting requested review from wearyzen

@d3zd3z d3zd3z Awaiting requested review from d3zd3z

@ithinuel ithinuel Awaiting requested review from ithinuel

@Vge0rge Vge0rge Awaiting requested review from Vge0rge

@cfriedt cfriedt Awaiting requested review from cfriedt

@kartben kartben Awaiting requested review from kartben

Assignees

@d3zd3z d3zd3z

@tomi-font tomi-font

Labels

area: TF-M ARM Trusted Firmware-M (TF-M) bug The issue is a bug, or the PR is fixing a bug manifest manifest-trusted-firmware-m Release Notes To be mentioned in the release notes size: XS A PR changing only a single line of code

Projects

None yet

Milestone

v4.3.0

Development

Successfully merging this pull request may close these issues.

5 participants

@nandojve @jhedberg @tomi-font @d3zd3z @zephyrbot